Move dependencies needed in CI on prod images to separate requirements file. #22898
+1,241
−617
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes: mozilla/addons#15191
Fixes: mozilla/addons#15066
Description
Key Changes
docker-compose.override.yml
(marked as obsolete).env
filemake
commandsolympia
user and permissionsContext
Impacts of this PR are:
make setup
to initialize their environment.env
file becomes crucial for configuration (automatically managed by make)make up
andmake down
handle most common operationsmake down
to clean up old configurations.env
file usingmake setup
Testing
TODO:
Verify make up fails when web doesn't start.
web
andworker
items in theProcfile
web
is unhealthyVerify site-static files are available in "prod" mode only
make up DOCKER_TARGET=development
/data/olympia/site-static
to be emptyVerify CI dependencies available only with flag set
make up INSTALL_CI_DEPS=True
./scripts/check_pip_packages.sh ci
First create an env in prod mode
Expect no ci or dev dependencies.
now run:
Now expect production and ci deps to be installed
FInally install dev deps
You can trigger these deps to be installed automatically if you set the argument on make up
This will install prod and ci deps in the container.
Checklist
#ISSUENUM
at the top of your PR to an existing open issue in the mozilla/addons repository.